-
Notifications
You must be signed in to change notification settings - Fork 5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Website Gallery page building #850
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #850 +/- ##
=======================================
Coverage 26.58% 26.58%
=======================================
Files 28 28
Lines 3732 3732
Branches 847 847
=======================================
Hits 992 992
Misses 2667 2667
Partials 73 73
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Does the page render as a gallery? |
@sonichi Now it is rendered using a markdown page. I am now trying to make it render as a gallery. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to add tags for each? It'll be hard to browse.
Hi @skzhang1 , my current intuition is that we will benefit from building a custom (reusable) react component that can format data as a gallery. I am not sure how to accomplish it yet, but can do some research and share my findings. |
My findings so far are as follows:
[
{
"title": "AutoGen Playground",
"link": "https://huggingface.co/spaces/thinkall/AutoGen_Playground",
"description": "A space to explore the capabilities of AutoGen.",
"image": "default.png",
"tags": ["web", "app"]
},
...
]
I have some sample code for this .. the result being below. @skzhang1 is it ok if I committed the code to this PR and then you can modify it as needed? let me know. @sonichi .. is this close to what you are envisioning? |
Yes. Looks great! |
@victordibia It looks great to me! Can you make a commit for that? We will enrich the content correspondingly. |
…rl state management
@skzhang1, I made a commit. Thanks for taking the lead and helping with this feature, it will really help the community Note: You might have to run |
@victordibia Thanks for your help! I made small changes to the content. Now this PR looks good to me. I will add icons to these applications in another PR. I think we can merge these PR and encourage uses to submit their demos in the community at first. Can you check the content again? @sonichi @qingyun-wu Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
I agree that it might be a good idea add some icons or color coding to each tag.
We might also want an author field to indicate if it is a community example or by the autogen team. And yes, let us work on that on a different PR.
Thanks again for working on this!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't go through all the examples. I checked a few and commented.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the case when image is not provided, I meant rendering the image contained in the linked page, like the gallery forum in discord: https://discord.com/channels/1153072414184452236/1161015724521836634
It it's hard to do it here, we can use a common default image as proposed, but the default image needs to be replaced by a more commonly representative image like the autogen landing image.
* initial draft * update dependences, add antd for layout (note yarn install needed her) * update gallery to use mdx file format to support react components * add custom react component for gallery with tag based filtering and url state management * add styling for gallery objects * add gallery data structure and default image * improve layout for gallerypage * revise Postgres demo * add default value when image is not provided * move semetic kernal to ecosystem * update tags * update tags * update ecosystem * update default fig * update tags * reformat --------- Co-authored-by: “skzhang1” <“shaokunzhang529@gmail.com”> Co-authored-by: Victor Dibia <victordibia@microsoft.com>
…s specified, then effectively zero-shot AutoML is used (microsoft#850) * Added an info reminding user that if no time_budget and no max_iter is specified, then effectively zero-shot AutoML is used * moved message to line 2818 * Update flaml/automl/automl.py Co-authored-by: Chi Wang <wang.chi@microsoft.com> * Update flaml/automl/automl.py Co-authored-by: Chi Wang <wang.chi@microsoft.com> Co-authored-by: Chi Wang <wang.chi@microsoft.com> Co-authored-by: Qingyun Wu <qingyun.wu@psu.edu>
* initial draft * update dependences, add antd for layout (note yarn install needed her) * update gallery to use mdx file format to support react components * add custom react component for gallery with tag based filtering and url state management * add styling for gallery objects * add gallery data structure and default image * improve layout for gallerypage * revise Postgres demo * add default value when image is not provided * move semetic kernal to ecosystem * update tags * update tags * update ecosystem * update default fig * update tags * reformat --------- Co-authored-by: “skzhang1” <“shaokunzhang529@gmail.com”> Co-authored-by: Victor Dibia <victordibia@microsoft.com>
Why are these changes needed?
There are many interesting examples and resources from discord, twitter, linkedin, medium, youtube, reddit, github.
We need to build a gallery page to collect these interesting examples and encourage the community to enrich the content.
Related issue number
Closes #830
Tasks
Checks